Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add harper 0.14.0 #6314

Merged
merged 8 commits into from
Jan 6, 2025
Merged

Add harper 0.14.0 #6314

merged 8 commits into from
Jan 6, 2025

Conversation

EphemeralDev
Copy link
Contributor

Closes #6313

initial json for harper-ls
remove v from version
Copy link
Contributor

github-actions bot commented Nov 9, 2024

Your changes do not pass checks.

harper

  • Description
  • License
  • Hashes
  • Checkver
  • Autoupdate

@EphemeralDev
Copy link
Contributor Author

Couple questions, should version in the json include the prefix 'v'. For example v0.12.0 .
Harper does have a website, should that be used as the homepage or the github repository?

removed regex and added 'v' before $version check in update url
@EphemeralDev
Copy link
Contributor Author

/verify

Copy link
Contributor

github-actions bot commented Nov 9, 2024

Your changes do not pass checks.

Invalid manifests

  • harper

removed extra comma
@EphemeralDev
Copy link
Contributor Author

/verify

Copy link
Contributor

All changes look good.

Wait for review from human collaborators.

harper

  • Description
  • License
  • Hashes
  • Checkver
  • Autoupdate

Updates homepage, download, checkver, and autoupdate urls
@elijah-potter
Copy link

Is there anything we can do to get this merged sooner?

@EphemeralDev
Copy link
Contributor Author

EphemeralDev commented Jan 3, 2025

Is there anything we can do to get this merged sooner?

I can ping a maintainer to see if it's good to go. Looking back at my PR (which i made rather hastily), should i make adjustments to the description (to clarify that its a language server) and/or install command (to differentiate between ls and cli)?

If it were to be merged now it would be called harper (scoop install harper), and install harper-ls. I only ask because harper cli exists as well, and maybe someone requests that in the future.

@elijah-potter
Copy link

If it were to be merged now it would be called harper (scoop install harper), and install harper-ls. I only ask because harper cli exists as well, and maybe someone requests that in the future.

I think you can keep the name, and install both binaries. I believe this is what the Arch package does.

@EphemeralDev EphemeralDev changed the title Add harper 0.12.0 Add harper 0.14.0 Jan 5, 2025
@EphemeralDev
Copy link
Contributor Author

@niheaven Could you review this PR when you get the chance?

@niheaven niheaven merged commit 12cbab3 into ScoopInstaller:master Jan 6, 2025
2 checks passed
@EphemeralDev EphemeralDev deleted the add-harper branch January 6, 2025 05:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Request]: Harper
3 participants